Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

containers-storage: add new option .fuse_program #191

Merged
merged 1 commit into from
Jun 29, 2018

Conversation

giuseppe
Copy link
Member

@giuseppe giuseppe commented Jun 25, 2018

When specified the mount of the overlay file system is delegated to
the specified command instead of mounting it directly.

This patch enables the usage of helpers programs for mounting overlay using a FUSE program.

Signed-off-by: Giuseppe Scrivano [email protected]

@giuseppe
Copy link
Member Author

/cc @rhatdan @nalind

cmdRootless.Dir = d.home
return cmdRootless.Run()
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra newline?

@nalind
Copy link
Member

nalind commented Jun 28, 2018

One nit, otherwise the patch looks fine.

@rhatdan
Copy link
Member

rhatdan commented Jun 28, 2018

LGTM, @giuseppe fix the nit and I will merge.

I also want to merge the ostree patch, ok with you @nalind ?

When specified the mount of the overlay file system is delegated to
the specified command instead of mounting it directly.

Signed-off-by: Giuseppe Scrivano <[email protected]>
@giuseppe giuseppe force-pushed the overlay-fuse-program branch from b349d1a to fc4b862 Compare June 29, 2018 07:14
@giuseppe
Copy link
Member Author

extra newline dropped.

@giuseppe giuseppe changed the title [RFC] containers-storage: add new option .fuse_program containers-storage: add new option .fuse_program Jun 29, 2018
@rhatdan rhatdan merged commit 58f557c into containers:master Jun 29, 2018
@runcom runcom mentioned this pull request Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants